Skip to content

build: 版整合の検査と同期をカタログの宣言に基づかせる - #52

Merged
k163377 merged 1 commit into
mainfrom
fix/version-check-uses-declared-version
Aug 29, 2026
Merged

k163377 merged 1 commit into
mainfrom
fix/version-check-uses-declared-version

Conversation

@k163377

@k163377 k163377 commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

背景

週次のマイナー横断ビルド(kotlin-prerelease)は、過去 3 回の定期実行がすべて
ルートビルドの最初の段で失敗している(run 33211294275 ほか 2 件)。

Execution failed for task ':checkVersionMentions'.
> 現行バージョンとズレた表記があります: docs/実装ノート.md, docs/test/テスト戦略.md,
  docs/エッジケースへの対応方針.md, .idea/kotlinc.xml

原因は版整合の機構が源とする値である。-PkotlinVersionOverride は settings が
カタログの解決値だけを差し替えるが、checkVersionMentions / checkMavenVersion は
libs.versions.kotlin すなわち差し替え後の値を読んでいた。宣言(gradle/libs.versions.toml)も
それを写した資料も差し替えでは変わらないため、資料は常に「乖離」と判定される。
このためプレリリースの検証は一度も本体のビルドへ到達していない。

syncVersionMentions にも同じ経路があり、差し替え中に実行すると宣言に無い版を資料へ書き込む。
checkMavenVersion はプレリリースの POM から maven 版を導いており、上流が
<maven.version> を上げた時点で同じく偽陽性となる。

変更

検査・同期が読む Kotlin / maven 版を、カタログのテキストから直接取る宣言版へ変更する。
一時的な差し替えの影響を受けず、検査の意味(宣言と資料の一致)はそのまま保たれる。

確認

  • ./gradlew ktfmtFormat checkVersionMentions checkMavenVersion → BUILD SUCCESSFUL
  • ./gradlew checkVersionMentions checkMavenVersion -PkotlinVersionOverride=2.4.20-RC2 → BUILD SUCCESSFUL
    (KGP は 2.4.20-RC2 を解決、kotlin-maven-plugin の POM は宣言版 2.4.10 のまま)

プレリリース本体(2.4.20-RC2 でのビルドとテスト)が通るかは、マージ後の kotlin-prerelease で判定する。

🤖 Generated with Claude Code

マイナー横断ビルド(kotlin-prerelease)は -PkotlinVersionOverride で
カタログの Kotlin 版の解決値を差し替える。checkVersionMentions と
checkMavenVersion がその解決値を源としていたため、宣言と一致した資料を
乖離と見なして必ず失敗し、プレリリースの検証はルートビルドの最初の段で
止まっていた(過去 3 回の定期実行がすべて同じ失敗)。

検査・同期が読む版を gradle/libs.versions.toml のテキストから取り、
一時的な差し替えの影響を受けないようにする。上流 POM の導出座標も
宣言版へ揃え、プレリリースの POM から maven 版を導かないようにする。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@k163377
k163377 marked this pull request as ready for review August 29, 2026 08:27
@k163377
k163377 merged commit dfb85c1 into main Aug 29, 2026
12 checks passed
@k163377
k163377 deleted the fix/version-check-uses-declared-version branch August 29, 2026 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant